<html>
<head><meta charset="utf-8"><title>lint file location clippy#6643 · clippy · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/index.html">clippy</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html">lint file location clippy#6643</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="225190086"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225190086" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Yusuke Tanaka <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225190086">(Feb 04 2021 at 17:05)</a>:</h4>
<p>Hi there!<br>
I'm working on <a href="https://github.com/rust-lang/rust-clippy/issues/6643">https://github.com/rust-lang/rust-clippy/issues/6643</a>. While I was browsing the repo to learn the directory structure, I found that most lint rules are placed in <code>clippy_lints/src</code> but some are in <code>clippy_lints/src/methods</code>. And I also read "Adding a new lint" saying that <code>cargo dev new_lint</code> command creates a new rule file in <code>clippy_lints/src</code>. So I'm sort of confused... why are some rules in <code>clippy_lints/src/methods</code>? How do we decide where to put a new rule file?</p>



<a name="225198540"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225198540" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> xFrednet <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225198540">(Feb 04 2021 at 18:06)</a>:</h4>
<p>Hey <span class="user-mention" data-user-id="371128">@Yusuke Tanaka</span> welcome to clippy :)<br>
<span class="user-mention silent" data-user-id="371128">Yusuke Tanaka</span> <a href="#narrow/stream/257328-clippy/topic/lint.20file.20location/near/225190086">said</a>:</p>
<blockquote>
<p>How do we decide where to put a new rule file?</p>
</blockquote>
<p>I usually first search for a similar lint and then add it to the same file. This is often easier and you can also reuse the structure of the other lint. An example for your case would be <a href="https://rust-lang.github.io/rust-clippy/master/#flat_map_identity">FLAT_MAP_IDENTITY</a> (<a href="https://github.com/rust-lang/rust-clippy/blob/357c6a7e2739da8c8d0a68a5e9e0867983af7cbe/clippy_lints/src/methods/mod.rs#L545">Code definition</a>)</p>
<p><span class="user-mention silent" data-user-id="371128">Yusuke Tanaka</span> <a href="#narrow/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643/near/225190086">said</a>:</p>
<blockquote>
<p>[...] I found that most lint rules are placed in <code>clippy_lints/src</code> but some are in <code>clippy_lints/src/methods</code>.</p>
</blockquote>
<p>I'm not sure about the origin of this structure as I'm also fairly new. My guess is that the <code>methods</code> module grew to the point were it couldn't be reasonably contained in one file.</p>



<a name="225229979"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225229979" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Nahua <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225229979">(Feb 04 2021 at 22:00)</a>:</h4>
<p>Hi :)</p>
<p>Besides the <code>src/methods/</code> directory which @xFrednet mentioned, some lints also reside in <code>clippy_lints/src/loops.rs</code> file (mostly for-loop related lints).</p>
<p>For example, the two lints I have created so far, <a href="https://rust-lang.github.io/rust-clippy/master/#manual_flatten">manual_flatten</a> and <a href="https://rust-lang.github.io/rust-clippy/master/#inspect_for_each">inspect_for_each</a>, don't work very smoothly with the <code>cargo dev new_lint</code>.</p>
<p>I just ran the command, then delete the files, copy the content and move the code to the relevant files (<code>src/loops.rs</code> or <code>src/methods/mod.rs</code>) and then imitate how other lints in these files are declared and where in the code they reside. Note that it's important to read some other lints and see how they are implemented. Imitate their setup for your new lint.</p>
<p>Then I update <code>CHANGELOG.md</code> and <code>src/lib.rs</code>so that they are hooked to the correct module for the new lint (<a href="https://github.com/rust-lang/rust-clippy/pull/6646">example PR here</a>).</p>
<p>Finally, I write some tests and start checking if the new lint is hooked properly by running , for example,<code>TESTNAME=manual_flatten cargo uitest</code> and see if I get errors in the console. If it does, congrats and now you can start working on the lint!</p>



<a name="225245915"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225245915" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Yusuke Tanaka <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225245915">(Feb 05 2021 at 00:33)</a>:</h4>
<p>Thanks so much <span class="user-mention" data-user-id="369415">@xFrednet</span> and <span class="user-mention" data-user-id="374205">@Nahua</span> !<br>
I see, then I'll follow the current convention that the existing rules are following, sometimes moving the result of <code>cargo dev new_lint</code> to somewhere appropriate <span aria-label="smile" class="emoji emoji-1f642" role="img" title="smile">:smile:</span> </p>
<blockquote>
<p>My guess is that the methods module grew to the point were it couldn't be reasonably contained in one file.</p>
</blockquote>
<blockquote>
<p>Besides the src/methods/ directory which @xFrednet mentioned, some lints also reside in clippy_lints/src/loops.rs file (mostly for-loop related lints).</p>
</blockquote>
<p>These make a lot of sense to me!</p>



<a name="225274017"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225274017" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> flip1995 <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225274017">(Feb 05 2021 at 08:58)</a>:</h4>
<blockquote>
<p>My guess is that the <code>methods</code> module grew to the point were it couldn't be reasonably contained in one file.</p>
</blockquote>
<p>Exactly. The <code>methods/mod.rs</code> file (previously in <code>clippy_lints/src/methods.rs</code>) is still way too big wiht multiple thousands of lines. Same goes for <code>loops.rs</code> <code>types.rs</code> and <code>transmute.rs</code> (maybe also <code>misc.rs</code>, I'm not really sure what's in that file though). We _should_ refactor clippy to adopt the structure that <code>methods</code> _should_ use. But refactoring isn't fun, so no one has done it.</p>
<blockquote>
<p>Then I update <code>CHANGELOG.md</code> and <code>src/lib.rs</code>so that they are hooked to the correct module for the new lint</p>
</blockquote>
<p><code>cargo dev update_lints</code> does that for you. Even when you move lints around/deprecate/rename them. So no need to do this by hand. (Except you implement a new <code>LintPass</code> then you have to add a line to <code>lib.rs</code>)</p>
<blockquote>
<p>Finally, I write some tests </p>
</blockquote>
<p>I always recommend doing this as a first step. It helps when writing the lint and it prevents you to just test what currently works. And you will have to write them anyway.</p>



<a name="225292304"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225292304" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Yusuke Tanaka <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225292304">(Feb 05 2021 at 12:29)</a>:</h4>
<p>Thanks for the additonal info, <span class="user-mention" data-user-id="264664">@flip1995</span> .  I totally get it!<br>
I like test-driven development too :)</p>
<blockquote>
<p>We _should_ refactor clippy to adopt the structure that methods _should_ use. But refactoring isn't fun, so no one has done it.</p>
</blockquote>
<p>I think I can help with it. May I work on it after <a href="https://github.com/rust-lang/rust/issues/6643">#6643</a> is done, and should I open a new issue about it?</p>



<a name="225298065"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225298065" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Nahua <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225298065">(Feb 05 2021 at 13:29)</a>:</h4>
<blockquote>
<p>I always recommend doing this as a first step. It helps when writing the lint and it prevents you to just test what currently works.</p>
</blockquote>
<p>Yes the "Finally" was meant to be the first thing after the <code>cargo dev new_lint</code> process is completed, before any logic for the lint is written. <span aria-label="+1" class="emoji emoji-1f44d" role="img" title="+1">:+1:</span> </p>
<blockquote>
<p><code>cargo dev update_lints</code> does that for you. Even when you move lints around/deprecate/rename them. So no need to do this by hand.</p>
</blockquote>
<p>Thanks! That's great to know :)</p>



<a name="225318367"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225318367" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225318367">(Feb 05 2021 at 16:07)</a>:</h4>
<blockquote>
<blockquote>
<p>We _should_ refactor clippy to adopt the structure that methods _should_ use. But refactoring isn't fun, so no one has done it.</p>
</blockquote>
<p>I think I can help with it. May I work on it after <a href="https://github.com/rust-lang/rust/issues/6643">#6643</a> is done, and should I open a new issue about it?</p>
</blockquote>
<p>Opening an issue would be good to track the work IMO <span aria-label="thumbs up" class="emoji emoji-1f44d" role="img" title="thumbs up">:thumbs_up:</span> </p>
<p>To see if we're on the same page, I think the refactor would look like this:</p>
<div class="codehilite" data-code-language="Diff"><pre><span></span><code><span class="gd">-["map", "filter"] =&gt; lint_filter_map(..),</span>
<span class="gi">+["map", "filter"] =&gt; filter_map::check(..),</span>
</code></pre></div>
<p>...where <code>filter_map</code> is a file/module containing a <code>declare_clippy_lint!</code> and a <code>check</code> method. A nit, but I would prefer the function name <code>check</code> over <code>lint</code> for functions that check but only sometimes lint.</p>



<a name="225318775"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225318775" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> flip1995 <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225318775">(Feb 05 2021 at 16:10)</a>:</h4>
<p>Yes, but I would keep the <code>declare_clippy_lint!</code> in the <code>mod.rs</code> Best case would be that <code>mod.rs</code> only contains the lint definitions and the LintPass impl. Utility functions of the module should probably go in <code>methods/utils.rs</code> and then reexported for the module in <code>mod.rs</code>.</p>



<a name="225319917"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225319917" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225319917">(Feb 05 2021 at 16:18)</a>:</h4>
<p>Wouldn't it be better to have the lint declaration, docs, and (most of) its implementation all in the same place?</p>



<a name="225321046"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225321046" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> flip1995 <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225321046">(Feb 05 2021 at 16:26)</a>:</h4>
<p>I think that is just preference. I like to keep all definitions in <code>mod.rs</code> and implementations in the linting files.</p>



<a name="225321526"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225321526" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225321526">(Feb 05 2021 at 16:30)</a>:</h4>
<p>okay</p>



<a name="225322827"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225322827" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Nahua <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225322827">(Feb 05 2021 at 16:39)</a>:</h4>
<p><span aria-label="+1" class="emoji emoji-1f44d" role="img" title="+1">:+1:</span> Would love to help out on this refactoring if we decide to do it!</p>



<a name="225323551"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225323551" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> flip1995 <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225323551">(Feb 05 2021 at 16:44)</a>:</h4>
<p>Go for it! But if you split lints out of <code>mod.rs</code> do it in at least one commit per lint. Otherwise this will be impossible to review.</p>



<a name="225324615"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225324615" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Yusuke Tanaka <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225324615">(Feb 05 2021 at 16:52)</a>:</h4>
<p><span class="user-mention silent" data-user-id="360405">Cameron Steffen</span> <a href="#narrow/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643/near/225318367">said</a>:</p>
<blockquote>
<p>Opening an issue would be good to track the work IMO <span aria-label="thumbs up" class="emoji emoji-1f44d" role="img" title="thumbs up">:thumbs_up:</span> </p>
</blockquote>
<p>Alright! I'll try to elaborate an issue.</p>
<p><span class="user-mention silent" data-user-id="374205">Nahua</span> <a href="#narrow/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643/near/225322827">said</a>:</p>
<blockquote>
<p><span aria-label="+1" class="emoji emoji-1f44d" role="img" title="+1">:+1:</span> Would love to help out on this refactoring if we decide to do it!</p>
</blockquote>
<p>Cool, I'd be happy if we could split the task :)</p>



<a name="225326323"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225326323" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225326323">(Feb 05 2021 at 17:03)</a>:</h4>
<p>You could split them alphabetically <span aria-label="laughing" class="emoji emoji-1f606" role="img" title="laughing">:laughing:</span></p>



<a name="225329115"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225329115" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Yusuke Tanaka <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225329115">(Feb 05 2021 at 17:24)</a>:</h4>
<p><a href="https://github.com/rust-lang/rust-clippy/issues/6680">https://github.com/rust-lang/rust-clippy/issues/6680</a><br>
Just opened it. Please let me know if there's anything to refine, because I'm very new to this repo <span aria-label="wink" class="emoji emoji-1f609" role="img" title="wink">:wink:</span></p>



<a name="225333598"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225333598" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225333598">(Feb 05 2021 at 17:58)</a>:</h4>
<p>I would add: For each lint, create a new module at <code>clippy_lints/src/&lt;group_name&gt;/&lt;lint_name&gt;.rs</code>. Within that module, add a function <code>fn check(cx, ..)</code> (with any needed arguments) and move the implementation there.</p>



<a name="225388226"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225388226" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Yusuke Tanaka <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225388226">(Feb 06 2021 at 05:29)</a>:</h4>
<p>Sounds good, I added it <span aria-label="+1" class="emoji emoji-1f44d" role="img" title="+1">:+1:</span></p>



<a name="225453513"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225453513" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Phil H <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225453513">(Feb 07 2021 at 09:01)</a>:</h4>
<p><span class="user-mention silent" data-user-id="360405">Cameron Steffen</span> <a href="#narrow/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643/near/225333598">said</a>:</p>
<blockquote>
<p>[..] <code>clippy_lints/src/&lt;group_name&gt;/&lt;lint_name&gt;.rs</code>. [..]</p>
</blockquote>
<p><del>Hmm, I probably wouldn't include the group name in the path, because we change lint groups quite often. Or at least more often than lint names</del><br>
nvm, will comment on <a href="https://github.com/rust-lang/rust-clippy/issues/6680">clippy#6680</a> instead <span aria-label="smile" class="emoji emoji-1f642" role="img" title="smile">:smile:</span></p>



<a name="225463671"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225463671" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Nahua <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225463671">(Feb 07 2021 at 13:15)</a>:</h4>
<p>A question before further refactoring commits:</p>
<p>In <code>loops.rs</code>, we have <code>check_for_loop_arg</code> which covers two lints: <code>explicit_into_iter_loop</code> and <code>iter_next_loop</code>, should we keep the lints together in a new file? And what should the convention for the file names be? Do we keep using <code>check_&lt;something&gt;</code> in this case? :)</p>



<a name="225564642"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225564642" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225564642">(Feb 08 2021 at 15:59)</a>:</h4>
<p>how bout <code>loops/for_loop_arg.rs</code></p>



<a name="225564870"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225564870" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Cameron Steffen <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225564870">(Feb 08 2021 at 16:00)</a>:</h4>
<p>with <code>check_for_loop_arg(..)</code></p>



<a name="225566681"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/257328-clippy/topic/lint%20file%20location%20clippy%236643/near/225566681" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Nahua <a href="https://rust-lang.github.io/zulip_archive/stream/257328-clippy/topic/lint.20file.20location.20clippy.236643.html#225566681">(Feb 08 2021 at 16:12)</a>:</h4>
<p>Yes, that is a reasonable approach. I can get started on that tonight :)</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>